Update agui.md#2274
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
AgentScopeJavaBot
left a comment
AgentScopeJavaBot
left a comment
There was a problem hiding this comment.
🤖 AI Review
This PR fixes a typo in the AG-UI integration doc where the Spring Boot starter artifact name was written as agentscope-spring-boot-starter-agui instead of the correct agentscope-agui-spring-boot-starter. The fix is correct — confirmed against the actual Maven module POM. However, the Chinese translation of the same document (docs/v2/zh/integration/protocol/agui.md, line 53) still contains the identical typo and should be fixed in the same PR for consistency.
| ## Spring Boot integration | ||
|
|
||
| The typical wire-up is a `@PostMapping("/ag-ui")` returning the `Flux<AguiEvent>` as SSE. Alternatively, use `agentscope-spring-boot-starter-agui` to register the controller automatically. | ||
| The typical wire-up is a `@PostMapping("/ag-ui")` returning the `Flux<AguiEvent>` as SSE. Alternatively, use `agentscope-agui-spring-boot-starter` to register the controller automatically. |
There was a problem hiding this comment.
[nit] Good catch — the artifact name is now correct. However, the Chinese counterpart docs/v2/zh/integration/protocol/agui.md (line 53) still reads agentscope-spring-boot-starter-agui and should be updated to agentscope-agui-spring-boot-starter in the same commit to keep both translations in sync.
AgentScopeJavaBot
left a comment
AgentScopeJavaBot
left a comment
There was a problem hiding this comment.
🤖 AI Review
This PR fixes a typo in the AG-UI integration doc where the Spring Boot starter artifact name was written as agentscope-spring-boot-starter-agui instead of the correct agentscope-agui-spring-boot-starter. The fix is correct — confirmed against the actual Maven module POM. However, the Chinese translation of the same document (docs/v2/zh/integration/protocol/agui.md, line 53) still contains the identical typo and should be fixed in the same PR for consistency.
| ## Spring Boot integration | ||
|
|
||
| The typical wire-up is a `@PostMapping("/ag-ui")` returning the `Flux<AguiEvent>` as SSE. Alternatively, use `agentscope-spring-boot-starter-agui` to register the controller automatically. | ||
| The typical wire-up is a `@PostMapping("/ag-ui")` returning the `Flux<AguiEvent>` as SSE. Alternatively, use `agentscope-agui-spring-boot-starter` to register the controller automatically. |
There was a problem hiding this comment.
[nit] Good catch — the artifact name is now correct. However, the Chinese counterpart docs/v2/zh/integration/protocol/agui.md (line 53) still reads agentscope-spring-boot-starter-agui and should be updated to agentscope-agui-spring-boot-starter in the same commit to keep both translations in sync.
oss-maintainer
left a comment
oss-maintainer
left a comment
There was a problem hiding this comment.
Summary
This PR contains 13 lines of changes. Review in progress.
Automated review by github-manager-bot
AgentScope-Java Version
[The version of AgentScope-Java you are working on, e.g. 1.0.12, check your pom.xml dependency version or run
mvn dependency:tree | grep agentscope-parent:pom(only mac/linux)]Description
[Please describe the background, purpose, changes made, and how to test this PR]
Checklist
Please check the following items before code is ready to be reviewed.
mvn spotless:applymvn test)